-
Notifications
You must be signed in to change notification settings - Fork 49
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(deps): update dependency api-platform/core to v3.2.9 #4129
Conversation
Branch automerge failureThis PR was configured for branch automerge. However, this is not possible, so it has been raised as a PR instead.
|
Failure may be related to api-platform/core#5961 |
Merge queue setting changed
5952ac9
to
0d09022
Compare
709b5ca
to
bd01ede
Compare
ba0fc2c
to
031015a
Compare
34344f1
to
520bff7
Compare
520bff7
to
aeb5335
Compare
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. ⚠ Warning: custom changes will be lost. |
Api-platform 3.2 introduced some breaking changes on the format of error responses. We fixed these BCs with #3985. Upstream, the BCs have now been fixed in api-platform/core#5974, hence I reverted our own changes as well. see commit 2bb6fbc |
Another issue was introduced with api-platform/core#5901 IRI-references without a specific example value provided by us now contain "https://example.com". This messes up the automatic payload generation for our API tests. It's an easy fix for the tests, as we can easily override the example values. However, this also has an effect on the generated OpenAPI output (see snapshot changes). Might be worth to look through to them to verify if we're still ok with this, or whether there's any upstream issue we want to trigger. |
ResponseHeaderBag->get only returns the first header for the key. But we want all headers for the key set-cookie. if the key is 'set-cookie', this function returns the values in an flat array. Else it returns all headers with the the cookies.
@@ -680,11 +693,12 @@ components: | |||
type: object | |||
Activity.jsonhal-read_ScheduleEntry.Activity: | |||
deprecated: false | |||
description: 'A piece of programme that will be carried out once or multiple times in a camp.' | |||
description: '' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I find it strange that the descriptions of the derived models are now missing,
but not of all derived models, but when then only of derived models.
For me it is ok if we make the update first and then try to fix the documentation again
⛔ Feature branch deployment currently inactive.If the PR is still open, you can add the |
This PR contains the following updates:
3.2.4
->3.2.9
Release Notes
api-platform/core (api-platform/core)
v3.2.9
Compare Source
What's Changed
Full Changelog: api-platform/core@v3.2.8...v3.2.9
v3.2.8
Compare Source
Bug fixes
v3.2.7
Compare Source
Symfony 7 support.
Bug fixes
v3.2.6
Compare Source
Bug fixes
To have errors backward compatible with 3.1, use:
New extension points are available using Errors with
rfc_7807_compliant_errors: true
such as Error provider and Error Resourcev3.2.5
Compare Source
Full Changelog: api-platform/core@v3.2.4...v3.2.5
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR has been generated by Mend Renovate. View repository job log here.